Skip to content

Comments

Add CI configuration#12

Merged
uellue merged 5 commits intomainfrom
add-ci
Mar 18, 2024
Merged

Add CI configuration#12
uellue merged 5 commits intomainfrom
add-ci

Conversation

@sk1p
Copy link
Member

@sk1p sk1p commented Mar 13, 2024

Fixes #10

I fixed some of the test cases already (mainly the rename minimize -> optimize), except for tests/test_overfocus.py::test_translation_ref, which still fails. Not sure what's wrong there - the test case or the code?

This PR also adds a pre-commit config with flake8 and pyupgrade, among others.

sk1p added 3 commits March 13, 2024 19:51
- Support Python 3.12
- Drop support for old Python (which is already being dropped by
  LiberTEM)
- Collect coverage during testing
- `minimize` -> `optimize`; move `method` parameter into `minimizer_kwargs`
- The `callback` now gets the current loss as fourth argument
@sk1p sk1p requested a review from uellue March 13, 2024 18:59
@uellue
Copy link
Member

uellue commented Mar 18, 2024

The test failure was introduced by TemGym/TemGym@e32a510

@codecov
Copy link

codecov bot commented Mar 18, 2024

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@uellue uellue merged commit 7e08ca8 into main Mar 18, 2024
@sk1p sk1p deleted the add-ci branch March 18, 2024 10:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add CI

2 participants